Skip to content

docs(pools): clarify profile updates don't propagate to idle pool browsers#354

Merged
masnwilliams merged 2 commits into
mainfrom
hypeship/document-pool-profile-update-behavior
May 12, 2026
Merged

docs(pools): clarify profile updates don't propagate to idle pool browsers#354
masnwilliams merged 2 commits into
mainfrom
hypeship/document-pool-profile-update-behavior

Conversation

@masnwilliams
Copy link
Copy Markdown
Contributor

@masnwilliams masnwilliams commented May 2, 2026

Summary

Adds a pools FAQ entry explaining that updating a profile's contents does not propagate to already-warmed idle browsers in a pool. Pool browsers load profile contents at fill time; the only way to force a pool to pick up new profile contents is to flush idle browsers (or browserPools.update with discard_all_idle: true).

Test plan

  • Mintlify preview renders the new FAQ entry under browsers/pools/faq.mdx

Note

Low Risk
Low risk: documentation-only change that clarifies expected pool behavior and how to force refreshed profile contents.

Overview
Adds a new FAQ entry to browsers/pools/faq.mdx clarifying that updating a profile’s contents does not propagate to already-warmed idle pool browsers.

Documents the recommended ways to pick up updated profile data: kernel.browserPools.flush() (to rebuild idle browsers via refill) or kernel.browserPools.update() with discard_all_idle: true.

Reviewed by Cursor Bugbot for commit a9e88c5. Bugbot is set up for automated code reviews on this repo. Configure here.

@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented May 2, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
Kernel 🟢 Ready View Preview May 2, 2026, 5:33 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@masnwilliams masnwilliams marked this pull request as ready for review May 12, 2026 20:42
@firetiger-agent
Copy link
Copy Markdown

Firetiger deploy monitoring skipped

This PR didn't match the auto-monitor filter configured on your GitHub connection:

Any PR that changes the kernel API. Monitor changes to API endpoints (packages/api/cmd/api/) and Temporal workflows (packages/api/lib/temporal) in the kernel repo

Reason: PR only modifies documentation (pools FAQ) with no changes to API endpoints or Temporal workflows.

To monitor this PR anyway, reply with @firetiger monitor this.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Risk assessment: Very Low.

This PR adds a single FAQ entry in browsers/pools/faq.mdx with no codepath, config, navigation, infrastructure, schema, auth, or security-boundary changes. The blast radius is limited to documentation copy for one browser pools page. I found no existing approvals and no CODEOWNERS file or requested codeowner reviewers, so I'm approving.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Risk assessment: Very Low

This PR only adds a short FAQ entry to browsers/pools/faq.mdx clarifying browser pool/profile behavior. The diff is documentation-only, limited to one page, with no changes to codepaths, generated API references, navigation/configuration, infrastructure, prompts, or shared systems. I also didn’t find a CODEOWNERS file requiring owner review, and there were no existing approvals to preserve.

Approved based on the current diff.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@masnwilliams masnwilliams merged commit 1122002 into main May 12, 2026
4 checks passed
@masnwilliams masnwilliams deleted the hypeship/document-pool-profile-update-behavior branch May 12, 2026 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant